• chore(engine): publish engine bases in ci #4649 : 3 dependent PRs (#4650 , #4652 , #4666 )
• chore: lockfile lefthook #4647
• fix(rivetkit): surface native sqlite kv errors to callers #4645 👈 (View in Graphite)
• fix(rivetkit): prevent sleep races during disconnect and db work #4644
• fix(rivetkit): isolate engine envoys and propagate startup failures #4643
• test(rivetkit): remove wasm sqlite-only fixtures and coverage #4642
• chore(sqlite-vfs): remove wasm sqlite packages and workspace wiring #4641
• chore(rivetkit): remove wasm sqlite runtime #4614
• fix(sqlite-native): restore native startup kv preload #4639
• fix(sqlite-native): delete metadata before chunk range #4638
• fix(sqlite-native): keep truncate cache coherent #4637
• fix(sqlite-vfs): use delete range for truncate cleanup #4636
• fix(sqlite-native): restore kv error hook #4635
• perf(sqlite-native): avoid cloning cached read chunks #4634
• perf(sqlite-native): gate kv operation labels #4633
• perf(sqlite-native): reuse read cache for partial writes #4632
• perf(sqlite-native): remove delete file existence probe #4631
• fix(sqlite-native): gate native read cache behind env var #4630
• fix(pegboard): isolate runner config dc lookup failures #4625 : 1 other dependent PR (#4626 )
• docs(pegboard): note runner config upsert split write risk #4624
• fix(pegboard): unpack global runner config keys correctly #4623
• fix(pegboard-gateway): enforce tunnel message state #4622
• fix(pegboard-runner): clear terminal tunnel routes #4621
• fix(pegboard): skip protocol version keys in runner pool backfill #4620
• fix(pegboard): create missing normal runner configs #4619
• fix(epoxy): reuse cached optimistic value #4618
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

PR Review: fix(rivetkit): surface native sqlite kv errors to callers

Overview: This PR improves debuggability of native SQLite KV failures by enriching I/O-related SQLite errors with the underlying KV channel error. The approach is clean and targeted.

Code Quality - Strengths: (1) enrichNativeDatabaseError has a never return type, ensuring TypeScript knows catch branches always terminate - important for the exec wrapper where result must be definitively assigned. (2) The deduplication guard prevents double-appending. (3) takeLastKvError uses optional chaining appropriately as a best-effort diagnostic path, per CLAUDE.md guidance. (4) Error message mutation preserves the original stack trace.

Minor Issues: (1) Incomplete test coverage - tests only exercise the run path; exec and query have no coverage. Consider adding a test for query to verify the pattern holds. (2) Narrow I/O error regex - the pattern may miss other SQLite I/O codes like SQLITE_IOERR_READ or disk full errors; a short comment on what the regex intends to catch would help future maintainers. (3) Unfilled PR description - the checklist and test description sections are blank.

Correctness: No issues found. enrichNativeDatabaseError always throws (never), so post-try/catch code is only reachable on success. The KV error is only attached when both the SQLite error is I/O-related AND a KV error exists. The optional takeLastKvError interface extension is non-breaking.

Summary: Clean, well-scoped fix. Two suggestions before merge: (1) add test coverage for exec or query paths, (2) add a brief comment on the regex scope. Everything else looks good.

Generated with Claude Code